Skip to content

C#: More C# 8 features#1392

Merged
hvitved merged 13 commits into
github:rc/1.21from
calumgrant:cs/cs8/static-using-null
Jun 5, 2019
Merged

C#: More C# 8 features#1392
hvitved merged 13 commits into
github:rc/1.21from
calumgrant:cs/cs8/static-using-null

Conversation

@calumgrant

Copy link
Copy Markdown
Contributor

Implement the remaining C# 8 features. (Missed feature freeze as it was queuing on the Language enum).

  • Null-coalescing assignment, x ??= y
  • Using declarations, using file = File.Open(...);
  • static modifiers on local functions

@calumgrant calumgrant added the C# label Jun 3, 2019
@calumgrant calumgrant added this to the 1.21.0 milestone Jun 3, 2019
@calumgrant calumgrant requested a review from hvitved June 3, 2019 14:03
@calumgrant calumgrant requested a review from a team as a code owner June 3, 2019 14:03
@calumgrant calumgrant force-pushed the cs/cs8/static-using-null branch from 3ea97d8 to 1da292a Compare June 3, 2019 16:43

@hvitved hvitved left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work! A few minor remarks.

Comment thread csharp/ql/test/library-tests/csharp8/NullCoalescingAssignment.ql
@@ -0,0 +1,2 @@
description: Support for null-coalescing assignment, static local functions and using declarations.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oxford comma.

Comment thread csharp/ql/test/library-tests/csharp8/UsingDeclarations.expected
Comment thread csharp/ql/src/semmle/code/csharp/Stmt.qll
@calumgrant calumgrant force-pushed the cs/cs8/static-using-null branch from 1da292a to 9678f8e Compare June 4, 2019 17:11
@starcke-semmle starcke-semmle changed the base branch from master to rc/1.21 June 4, 2019 18:52
@calumgrant calumgrant force-pushed the cs/cs8/static-using-null branch from c88359b to 98ecfd7 Compare June 5, 2019 08:44
@calumgrant calumgrant requested a review from a team as a code owner June 5, 2019 08:44
@calumgrant calumgrant force-pushed the cs/cs8/static-using-null branch from 98ecfd7 to a6ecb42 Compare June 5, 2019 08:50
@calumgrant calumgrant requested a review from a team as a code owner June 5, 2019 08:50
Comment thread csharp/ql/test/library-tests/assignables/Assignables.cs
@calumgrant calumgrant force-pushed the cs/cs8/static-using-null branch from a6ecb42 to c88359b Compare June 5, 2019 09:04
@hvitved hvitved merged commit 51d6858 into github:rc/1.21 Jun 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants